Skip to content

docs(rules): add rules and remove old plan and actions#3372

Merged
KATO-Hiro merged 1 commit into
stagingfrom
#3364
Apr 9, 2026
Merged

docs(rules): add rules and remove old plan and actions#3372
KATO-Hiro merged 1 commit into
stagingfrom
#3364

Conversation

@KATO-Hiro
Copy link
Copy Markdown
Collaborator

@KATO-Hiro KATO-Hiro commented Apr 9, 2026

close #3364

Summary by CodeRabbit

リリースノート

  • Documentation

    • GitHub ActionsおよびPrismaデータベース管理に関する新しいルール文書を追加しました
  • Chores

    • 開発用の一度限りのマイグレーション解決ワークフローおよび関連する開発ノートドキュメントを削除しました

Document that workflow_dispatch only triggers from the default branch,
and add Prisma rules for @@Map table naming in manual SQL and P3009 recovery steps.
Remove one-time resolve-migration workflow and plan doc after migration fix confirmed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2d42e6d8-3d0d-45ca-9d3a-1f74ae69b4af

📥 Commits

Reviewing files that changed from the base of the PR and between 2b6e676 and 9680f90.

📒 Files selected for processing (4)
  • .claude/rules/github-actions.md
  • .claude/rules/prisma-db.md
  • .github/workflows/resolve-migration.yml
  • docs/dev-notes/2026-04-09/fix-vote-grade-check-constraints/plan.md
💤 Files with no reviewable changes (2)
  • .github/workflows/resolve-migration.yml
  • docs/dev-notes/2026-04-09/fix-vote-grade-check-constraints/plan.md

📝 Walkthrough

Walkthrough

GitHub Actionsワークフロー(マイグレーション解決用)と関連ドキュメント(実装計画)を削除し、一般的なGitHub ActionsおよびPrismaデータベースルールのドキュメントを追加・更新しました。ワンタイム運用手続きを標準化されたルールドキュメントに統合しています。

Changes

Cohort / File(s) Summary
ルール・ドキュメント整備
.claude/rules/github-actions.md, .claude/rules/prisma-db.md
GitHub ActionsのworkflowDispatch ポリシーおよびPrismaマイグレーションエラー P3009 復旧手順を標準ルールとして文書化。
ワンタイムワークフロー削除
.github/workflows/resolve-migration.yml
特定マイグレーション(20260406112057_add_vote_grade_check_constraints)解決用ワークフローを削除。
実装ノート削除
docs/dev-notes/2026-04-09/fix-vote-grade-check-constraints/plan.md
ステージング環境マイグレーション復旧手順を文書化したノートを削除(標準ルールに統合)。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

📋 ワンタイムの手続きを、
ルールの棚へしまい込み、
ドキュメント整理、スッキリと。
運用知恵は代々へ💡

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR タイトルは変更内容(ルール追加とプラン・ワークフロー削除)を簡潔かつ正確に反映している。
Linked Issues check ✅ Passed 変更内容が #3364 のレビュー指摘を反映:ルール文書化、レビューツール対応の一環として計画的に実施されている。
Out of Scope Changes check ✅ Passed 全変更が #3364 の範囲内。ワークフロー削除・計画文書削除・ルール追加は関連する規範整理として一貫性がある。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch #3364

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Collaborator Author

@KATO-Hiro KATO-Hiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@KATO-Hiro KATO-Hiro merged commit 71c7775 into staging Apr 9, 2026
3 checks passed
@KATO-Hiro KATO-Hiro deleted the #3364 branch April 9, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] レビューツールからの指摘事項のうち妥当だと判断したものを反映させましょう

1 participant